Skip to content

Conversation

@sadiq1971
Copy link
Member

No description provided.

Copy link
Member

@dhyaniarun1993 dhyaniarun1993 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sadiq1971 Have you removed the len(sources) >= 10 check. If yes, Is it working properly without it?

@sadiq1971
Copy link
Member Author

@sadiq1971 Have you removed the len(sources) >= 10 check. If yes, Is it working properly without it?

Yes I removed the check.
It's working but the number of stack traces is very high. But it make sense for json report.
I also have the sorting implemented while traversing in dfs so results should be consistent

@sadiq1971
Copy link
Member Author

@sadiq1971 Have you removed the len(sources) >= 10 check. If yes, Is it working properly without it?

Yes I removed the check. It's working but the number of stack traces is very high. But it make sense for json report. I also have the sorting implemented while traversing in dfs so results should be consistent

Ahh, I just noticed the mistake in the if condition. Let me check with it

@sadiq1971
Copy link
Member Author

@dhyaniarun1993 updated the code.
This implementation contains unique path for SDK packages and also breaking down entry point functions to lower level while building the stack trace.

@sadiq1971 sadiq1971 merged commit 949d643 into dev Mar 10, 2025
2 checks passed
@sadiq1971 sadiq1971 deleted the feat/multiple-paths branch March 10, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants